Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

seed-cache-review-tags-api #159

Merged
merged 5 commits into from
Sep 18, 2024
Merged

seed-cache-review-tags-api #159

merged 5 commits into from
Sep 18, 2024

Conversation

karthik8239
Copy link
Contributor

@karthik8239 karthik8239 commented Sep 16, 2024

Seed Cache Update and Review-tags api

  • Updated the seed cache to load the review tags from the database schema
  • Updated general memory storage to store review tags
  • Implements new API endpoint for retrieving review tags
  • Secures endpoint with admin authentication
  • Fetches review tags from application's memory storage
  • Returns array of ReviewTag objects matching database schema
  • Enables efficient access to review tags for admin users

Resolved #151

@devshahofficial
Copy link
Member

added @amandesai01 as a reviewer!

@amandesai01
Copy link
Member

link issue.

Copy link
Member

@amandesai01 amandesai01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete all the comments

@@ -12,6 +12,10 @@ export async function seedCache() {
.from(metaDataTable)
.where(inArray(metaDataTable.key, ['seoConfig', 'careerSiteConfig', 'firstSetupAccessKey']));

//getting the review tags from the reviewTagsTable
const reviewTags = await db.select().from(reviewTagsTable);
console.log(reviewTags); //debug to check the reviewtags response
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@amandesai01 you mean the comments or code?

Copy link
Contributor Author

@karthik8239 karthik8239 Sep 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed the comments and logging

server/api/review-tags/index.get.ts Show resolved Hide resolved
server/api/review-tags/index.get.ts Show resolved Hide resolved
server/utils/tasks/seed-cache.ts Outdated Show resolved Hide resolved
@amandesai01
Copy link
Member

@karthik8239 please wait, I am finishing up other tasks. Will take care of this.

@karthik8239
Copy link
Contributor Author

@karthik8239 please wait, I am finishing up other tasks. Will take care of this.

@amandesai01 Please let me know if I need to I will complete this

@amandesai01 amandesai01 merged commit cb24d48 into profilecity:main Sep 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose via API endpoint
3 participants